Feat: Implement CheckPatternBasic Algorithm #218
Conversation
mulmuri
changed the title
|
type Spec struct {
ID string
InputType string
}spec은 어떻게 작성하면 될까요? |
ID : check_pattern_basic 으로 부탁드립니다! |
| "github.com/stretchr/testify/assert" | ||
| ) | ||
|
|
||
| func TestXxx(t *testing.T) { |
There was a problem hiding this comment.
테스트 코드 이름에 테스트 대상이 명확히 드러나도록 수정해주세요
| func TestXxx(t *testing.T) { | |
| func TestModel(t *testing.T) |
There was a problem hiding this comment.
파일명을 example이 아니라 내용물이 잘 드러나도록 수정해주시기 바랍니다.
ex)model
There was a problem hiding this comment.
Job 구현이 깔끔하고 좋네요. 앞으로 새로운 job을 추가하실 때도 이처럼 하시면 됩니다.
There was a problem hiding this comment.
이미 알고리즘을 테스트 하셨지만, job이 데이터를 잘 처리하는지 보기 위해 간단한 테스트만 추가해주세요. 여기 테스트는 알고리즘 테스트케이스만큼 많을 필요는 없습니다.
앞으로 analyzer의 ID는 스네이크 케이스로 하실 건가요? |
석진님이 정책 결정해주세요 |
|
급하게 PR 올려서 수정할 부분이 많네요 ㅎㅎ 일단 요거 참고해서 통합테스트 진행해주세요 저녁에 일괄적으로 수정하겠습니다 |
|
네, 그러면 일단 테스트 관련 작업부터 진행하겠습니다. |
|
수정 다 끝나면 이PR은 추가 리뷰 없이 bypass로 merge하셔도 됩니다. |
변경 사항
진행상황
원래는 Telegraf를 붙일 계획이었으나, Telegraf는 실시간성이 부족해 적절하지 못하다는 판단에 kapacitor로 변경
그러나 kapacitor은 무지하게 복잡하고 어려움.. Kapacitor을 붙여서 influxdb to influxdb 파이프라인을 만드려 했으나 실패, kapacitor 부분 걷어내고 알고리즘만 PR에 포함
input.txt 및 outout.txt 위 방법으로 자동으로 뽑아내려 했으나 실패, 수동으로 작성할 예정
이후 작업요청
input.txt와 output.txt가 influxdb string protocol대로 되어있다는 가정하에 통합테스트코드 작성 부탁드립니다. 오늘 저녁에 수동으로 만들어서 추가하겠습니다.